Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Исправляет описание работы с worker #5564

Merged
merged 3 commits into from
Dec 1, 2024

Conversation

vladislav149
Copy link
Contributor

Описание

исправил описание работы с worker

Чек-лист

  • Текст оформлен согласно руководству по стилю
  • Ссылки на внутренние материалы начинаются со слеша и заканчиваются слэшем либо якорем на заголовок (/css/color/, /tools/json/, /tools/gulp/#kak-ponyat)
  • Ссылки на картинки, видео и демки относительные (images/example.png, demos/example/, ../demos/example/)

@github-actions github-actions bot added js Контент по JavaScript статья Расширенный материал labels Nov 25, 2024
@vitya-ne vitya-ne added the ошибка Что-то критичное и мешает label Nov 26, 2024
@vitya-ne
Copy link
Contributor

Привет,
спасибо за исправление.
Добавь, пожалуйста такую же правку в примере ниже:

worker.onmessage = function (e) { // Слушаем сообщения из воркера
  console.log(e.data) // <--
  // База, это 415-ый, как слышно?
}

@vladislav149
Copy link
Contributor Author

Привет, спасибо за исправление. Добавь, пожалуйста такую же правку в примере ниже:

worker.onmessage = function (e) { // Слушаем сообщения из воркера
  console.log(e.data) // <--
  // База, это 415-ый, как слышно?
}

исправил

@solarrust solarrust changed the title fix worker Исправляет описание работы с worker Nov 28, 2024
@@ -55,7 +55,7 @@ worker.postMessage({ message: '415-ый, я база, ответьте' })
```js
// Воркер: worker.js
onmessage = function (e) { // Слушаем сообщения из основного потока
if (e.message === '415-ый, я база, ответьте') {
if (e.data.message === '415-ый, я база, ответьте') {
{/* Отправляем сообщение из воркера в основной поток */}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Привет! Всё круто. Но тут по пути, строчкой ниже твоей правки, я заметил одну микро-бяку, давай её поправим заодно? Заменим на классический однострочный комментарий, как в остальной статье:

Suggested change
{/* Отправляем сообщение из воркера в основной поток */}
// Отправляем сообщение из воркера в основной поток

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Привет, поправил!

Copy link

Превью контента из bf0627b опубликовано.

Copy link
Member

@HellSquirrel HellSquirrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Спасибо!

@HellSquirrel HellSquirrel merged commit eaccfa8 into doka-guide:main Dec 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
js Контент по JavaScript ошибка Что-то критичное и мешает статья Расширенный материал
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants